Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ignore_refs parameter #48

Closed
wants to merge 7 commits into from
Closed

Add ignore_refs parameter #48

wants to merge 7 commits into from

Conversation

emcfarlane
Copy link
Collaborator

@emcfarlane emcfarlane commented Jul 9, 2024

This is an alternative to #44. Adds a new param ignore_refs to skip refs that match glob expressions. This defaults to dependabot/* to ignore dependabot runs which don't have access to secrets.

View the example workflow here: https://github.com/bufbuild/buf-action-demo/pull/13

Adds a new param `ignore_refs` to skip refs that match glob
expressions. This defaults to `dependabot/*` to ignore dependabot runs
which don't have access to secrets.

This comment has been minimized.

README.md Outdated Show resolved Hide resolved
@@ -22,12 +22,14 @@ import {

// Inputs are the inputs to the action, matching the inputs in the action.yml.
export interface Inputs {
ref_name: string;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this and why is it here? this is not a named option in action.yml

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its resolved from the inputs, not a parameter to the action.

Co-authored-by: Nick Snyder <[email protected]>
@nicksnyder
Copy link
Member

closing in favor of #44

@nicksnyder nicksnyder closed this Jul 15, 2024
@nicksnyder nicksnyder deleted the ed/ignoreBranches branch July 15, 2024 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants